-
Notifications
You must be signed in to change notification settings - Fork 203
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bug 6155: blShowSorting used for frontend user sorting #431
Bug 6155: blShowSorting used for frontend user sorting #431
Conversation
…r in the frontend
Hi Preethi, thanks for your work, but there are several tests failing. Can you have a look at them? Thank you! |
@sebbbbauer I have made the necessary changes to the Pull request's Unit test file. Kindly check if clears all the required tests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comments in the code should be removed.
Disabling blShowSorting not considered in details locator with category default sorting When the option blShowSorting is disabled and a default sorting is set to a category the details locator of the products has no ongoing article positions. - blShowSorting is set in the backend only to allow/deny sorting by user in the frontend Conflicts: source/Application/Component/Locator.php - Modified the unit test file for the corresponding pull request Conflicts: tests/Unit/Application/Controller/LocatorTest.php Closes #431
Disabling blShowSorting not considered in details locator with category default sorting When the option blShowSorting is disabled and a default sorting is set to a category the details locator of the products has no ongoing article positions. - blShowSorting is set in the backend only to allow/deny sorting by user in the frontend Conflicts: source/Application/Component/Locator.php - Modified the unit test file for the corresponding pull request Conflicts: tests/Unit/Application/Controller/LocatorTest.php Closes #431
Disabling blShowSorting not considered in details locator with category default sorting When the option blShowSorting is disabled and a default sorting is set to a category the details locator of the products has no ongoing article positions. - blShowSorting is set in the backend only to allow/deny sorting by user in the frontend Conflicts: source/Application/Component/Locator.php - Modified the unit test file for the corresponding pull request Conflicts: tests/Unit/Application/Controller/LocatorTest.php Closes #431
Thanks, the pr was merged to master recently. |
blShowSorting is set in the backend only to allow/deny sorting by user in the frontend